Skip to content

Test and Index fix#682

Merged
NeptuneHub merged 9 commits into
mainfrom
devel
Jun 23, 2026
Merged

Test and Index fix#682
NeptuneHub merged 9 commits into
mainfrom
devel

Conversation

@NeptuneHub

@NeptuneHub NeptuneHub commented Jun 22, 2026

Copy link
Copy Markdown
Owner

Test and Index fix, more in details:

  • UI/Accessibility: Committed in 82f70b6, never reverted, the button convergence and a11y improvements are live.
  • CI (mypy, coverage, Trivy): Added in ed87742, hardened in 62a7700.
  • Dependency bumps: mistralai pinned, pywin32 pinned, onnx -> 1.21.0, PyJWT -> 2.13.0.
  • paged_ivf optimization: zero-copy unpack_cell, numpy-int probe, mmap dedup cleanup

PR test builds:

NeptuneHub and others added 2 commits June 22, 2026 18:14
Accessibility:
- Global :focus-visible keyboard focus ring (style.css; white variant in sidebar via menu.css)
- Nav toggles: hamburger and "Administration" submenu get aria-label/aria-expanded/aria-controls;
  Administration is now a <button>; arrow marked aria-hidden
- Sidebar focus management: focus moves in on open, Esc closes and returns focus, closed
  off-screen sidebar marked inert + aria-hidden (no longer tabbable)
- aria-live status regions + role=progressbar(aria-valuenow) for long-task/search feedback
  (index, similarity, chat)
- Accessible combobox keyboard nav for the song autocomplete (similarity, map): Up/Down/Enter/Esc,
  combobox/listbox/option roles, aria-activedescendant

Button system convergence:
- Make .btn a drop-in (0.75rem 1.5rem); add .btn-success and .btn-block utilities
- Migrate every primary-action button across 13 templates to .btn .btn-{primary,danger,success} .btn-block
- Remove duplicated per-page .danger-button/.active-button blocks, the per-ID color selector lists,
  the !important green, and the brittle button[style*="#16A34A"] attribute selector
- Note: artist-similarity "Create Playlist" button is now green (success) for consistency with all
  other "Create Playlist on Media Server" buttons

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a shared rate limiter using flask-limiter to protect authentication and user administration endpoints, and implements a versioned, ordered database migration framework. It also enhances frontend accessibility with improved keyboard navigation, ARIA attributes, and focus styles, while refactoring button styles into shared utility classes. Feedback on these changes highlights a potential startup crash if the rate limit configuration strings are empty, suggesting a check to ensure they are truthy before applying the decorator.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread app_auth.py Outdated
@github-advanced-security

Copy link
Copy Markdown
Contributor

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@NeptuneHub NeptuneHub merged commit 15eb4b6 into main Jun 23, 2026
28 checks passed
@NeptuneHub NeptuneHub deleted the devel branch June 23, 2026 10:23
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants